feat: add nvim-tree and neotree integration #22
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ThomasK33
force-pushed
the
thomask33/nvim-tree-integration
branch
from
993fc6d to
9952bd4
Compare
Change-Id: I22a048a068ce66e6f6ee8c4177a2a78178ab6616 Signed-off-by: Thomas Kosiewski <[email protected]>
Enable diff tool to handle file creation by displaying empty buffer comparison for non-existent files instead of erroring. Includes automatic parent directory creation and updated test expectations. Change-Id: I0032117b04309c63b605e21390083abb9ec741b2 Signed-off-by: Thomas Kosiewski <[email protected]>
- Replace fallback to single-file selection with line-by-line node mapping - Support visual line selection (V), character selection (v), and block selection (Ctrl-V) - Add comprehensive test suite for nvim-tree visual selection scenarios - Include deduplication and root-level file filtering - Maintain compatibility with existing nvim-tree marks functionality Fixes issue where multi-selection in nvim-tree only sent cursor file instead of all selected files. Change-Id: Ida7b3154f0b6749903ff0e847752d5c263a09de7 Signed-off-by: Thomas Kosiewski <[email protected]>
ThomasK33
force-pushed
the
thomask33/nvim-tree-integration
branch
from
9952bd4 to
1ef01d3
Compare
|
Shouldn't we have just a regular |
- Extract broadcast_at_mention() for single file broadcasts - Extract add_paths_to_claude() for batch operations with optional delays - Refactor all existing handlers to use centralized functions - Add ClaudeCodeAdd <file-path> command for direct file addition - Remove verbose comments that describe "what" instead of "why" - Add comprehensive documentation to README with examples - Add 13 test cases covering command registration, validation, and functionality - Maintain all existing functionality and test compatibility (164 tests passing) Change-Id: I171dbc0bcc82d72c5c9a4f3d891a71fc3c814dbc Signed-off-by: Thomas Kosiewski <[email protected]>
Great point. I'll add both direct integrations and a |
- Add optional start-line and end-line arguments to ClaudeCodeAdd command - Convert 1-indexed user input to 0-indexed for Claude compatibility - Add comprehensive validation for line numbers (positive, start <= end) - Ignore line numbers for directories with debug logging - Remove redundant vim.notify calls (logger handles notifications) - Update README with line range syntax and examples - Add extensive test coverage for all line range scenarios - Clean up chatty comments that describe obvious operations Change-Id: Ied866de821628c6f274faac3c82e527a0eaf46e1 Signed-off-by: Thomas Kosiewski <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fixes #14
Summary
Features
:ClaudeCodeTreeAdd@-mention Support: Add @filename or @directory/ reference to files and folders using the new commandTest Coverage
Change-Id: I22a048a068ce66e6f6ee8c4177a2a78178ab6616